Skip to content

ARTEMIS-5972: Replace JNI with Panama Foreign Function & Memory (FFM)…#6444

Open
mayankkunwar wants to merge 1 commit into
apache:mainfrom
mayankkunwar:ARTEMIS-5972
Open

ARTEMIS-5972: Replace JNI with Panama Foreign Function & Memory (FFM)…#6444
mayankkunwar wants to merge 1 commit into
apache:mainfrom
mayankkunwar:ARTEMIS-5972

Conversation

@mayankkunwar

Copy link
Copy Markdown

… API for Journal Native Layer

@clebertsuconic

Copy link
Copy Markdown
Contributor

nice one @mayankkunwar

@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 2 times, most recently from 17ca830 to 758dfd6 Compare May 14, 2026 00:05
Comment thread pom.xml Outdated
@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 8 times, most recently from a91e37f to f177963 Compare May 20, 2026 18:01
Comment thread artemis-ffm/etc/checkstyle-suppressions.xml Outdated
Comment thread artemis-ffm/pom.xml Outdated
@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 7 times, most recently from 8ddbb55 to 8bcceee Compare June 2, 2026 17:25
@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 4 times, most recently from bda5293 to fbd575c Compare June 2, 2026 20:28
@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 10 times, most recently from b602e56 to 9fc249c Compare June 5, 2026 17:07
@clebertsuconic clebertsuconic force-pushed the ARTEMIS-5972 branch 2 times, most recently from beabca5 to a89db8c Compare June 5, 2026 20:24
@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 6 times, most recently from 017c72c to 571ebec Compare June 8, 2026 08:28
Comment thread artemis-distribution/pom.xml Outdated
</build>

<profiles>
<profile>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that you made the optional compilation in the ffm module, this could always be present, right?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just need to link artemis-ffm changes to artemis-journal, and after that we can always have it.

Comment thread artemis-journal/pom.xml Outdated
</activation>
<dependencies>
<dependency>
<groupId>org.apache.artemis</groupId>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this could always be present?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, let me push the linking part to artemis-journal, and after this we can.

Comment thread artemis-journal/pom.xml
<artifactId>maven-compiler-plugin</artifactId>
<executions>
<execution>
<id>java24-compile</id>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, we will need this as latest changes has FFM related component like MemorySegment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if we changed the signature to use ByteBuffer (at least what's exposed to this level?)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually.. never mind... the implementation of the SequentialFactory itself is dependent on other internal parts. I think we need it.

<build>
<plugins>
<plugin>
<artifactId>maven-compiler-plugin</artifactId>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess for this case we do, but just double checking with you.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, we will need this part as well

@clebertsuconic

Copy link
Copy Markdown
Contributor

having the optional compilation in FFM simplifies thing a lot... nice one

@mayankkunwar mayankkunwar force-pushed the ARTEMIS-5972 branch 5 times, most recently from 18075b2 to 5c113de Compare June 10, 2026 04:35
…, which is now introducing libaio.

In the configuration we are naming it as ASYNCIO_2.

Co-Authored-By: Clebert Suconic <clebertsuconic@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants